Skip to content

CNDE-2845 : added table name in helmcharts for debezium for dts1#1095

Open
jayasudhasundaram wants to merge 1 commit intomainfrom
CNDE-2845_nrt_odse_lookup_question
Open

CNDE-2845 : added table name in helmcharts for debezium for dts1#1095
jayasudhasundaram wants to merge 1 commit intomainfrom
CNDE-2845_nrt_odse_lookup_question

Conversation

@jayasudhasundaram
Copy link
Collaborator

@jayasudhasundaram jayasudhasundaram commented Jun 16, 2025

Description

Added the table entry for Covid Lab Postprocessing

Creating a new Helm chart?

  1. Does the service require an ingress?
    • Kubernetes Ingress resource are PATH based and only handled by modernization-api and dataingestion-service helm charts.
    • Currently, names of Kubernetes services have to be predictable if an ingress needs to point to them
  2. Chart directory structure (specific environment values.yaml files are allowed at the same level as values.yaml) 
    |── charts
    ├── new-helm-chart
        ├── templates
        |   ├── tests
        |   ├── helpers.tpl
        |   ├── *.yaml
        |   └─ Chart.yaml
        ├── values.yaml
        └─  README.md
  3. Do not include secret values anywhere in a Helm chart, take special care when creating values.yaml
  4. values.yaml is annotated to include parameter description and format as appropriate.
    • values.yaml is considered the production/default parameter file and should point to publically available container registries, if the service is publically available.

Updating a Helm Chart?

  1. Ensure no secrets have been committed.
  2. Ensure updates to existing Helm charts follow the guidelines contained in section Creating a new Helm chart.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sveselev sveselev Awaiting requested review from sveselev

@alog-enquizit alog-enquizit Awaiting requested review from alog-enquizit

@szamfir szamfir Awaiting requested review from szamfir

@pvijay-enquizit pvijay-enquizit Awaiting requested review from pvijay-enquizit

@njaved njaved Awaiting requested review from njaved

@cmoss-enq cmoss-enq Awaiting requested review from cmoss-enq

@tukhi tukhi Awaiting requested review from tukhi

@kramakur kramakur Awaiting requested review from kramakur

@akhamraev akhamraev Awaiting requested review from akhamraev

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@jayasudhasundaram